Skip to content

merge latest upstream main - #11

Open
dray92 wants to merge 408 commits into
devzero-inc:mainfrom
GoogleCloudPlatform:main
Open

dray92 wants to merge 408 commits into
devzero-inc:mainfrom
GoogleCloudPlatform:main

Conversation

@dray92

@dray92 dray92 commented Jan 13, 2025

Copy link
Copy Markdown

Background

Fixes

Change Summary

Additional Notes

Testing Procedure

Related PRs or Issues

Summary by CodeRabbit

  • New Features

    • Multi-architecture container builds; load generator adds RATE env (default "1").
  • Documentation

    • New microservice onboarding guide; clarified development guide; added demo links and minor README fixes.
  • Chores

    • CI/tooling and runtime upgrades (actions, .NET/Go versions); Terraform/provider version bumps; broad dependency and image updates; Skaffold multi-platform builds.
  • Resource Defaults

    • Increased memory defaults for cart, currency, and payment services.
  • Helm / Kubernetes

    • seccompProfile now gated by securityContext flag; many image tags, ports, probes, and health wiring updated.
  • Observability

    • OpenTelemetry tracing pipeline modernized; profiler startup disabled in some services.

@height

height Bot commented Jan 13, 2025

Copy link
Copy Markdown

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

@kevin-devzero

Copy link
Copy Markdown

closing for now

renovate-bot and others added 21 commits August 18, 2025 10:48
* Update README.md

* Update README.md

---------

Co-authored-by: Olivier Bourgeois <3271352+bourgeoisor@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Sep 22, 2025

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

Broad version and tooling upgrades across CI, runtimes, container images, and dependencies; many Dockerfiles made multi‑platform; Terraform provider/module constraints bumped; Helm templates adjusted to nest seccompProfile under securityContext; OpenTelemetry initialization switched to NodeSDK in Node services; cartstore uses an upsert; loadgenerator RATE env added.

Changes

Cohort / File(s) Summary
CI workflows & installer
​.github/workflows/ci-main.yaml, ​.github/workflows/ci-pr.yaml, ​.github/workflows/helm-chart-ci.yaml, ​.github/workflows/kubevious-manifests-ci.yaml, ​.github/workflows/kustomize-build-ci.yaml, ​.github/workflows/terraform-validate-ci.yaml, ​.github/workflows/helm-chart-ci.yaml, ​.github/workflows/install-dependencies.sh
Bumped GitHub Action usages (e.g., actions/checkout@v4@v6), upgraded setup actions and runtime targets (dotnet 9→10, Go 1.23→1.25), and updated install script.
Terraform & providers
​.github/terraform/main.tf, terraform/main.tf, terraform/providers.tf, ​.github/terraform/versions.tf
Module version bumps (enable_google_apis >17→>18, gcloud >3→>4) and Google provider constraint bumped (>6.x→>7.x).
Helm chart & templates
helm-chart/Chart.yaml, helm-chart/values.yaml, helm-chart/templates/*.yaml
Chart/appVersion → 0.10.4; increased cart/currency/payment memory; moved seccompProfile rendering inside .Values.securityContext.enable across templates; updated image digests and Otel images; loadgenerator RATE env added.
Kustomize & release manifests
release/kubernetes-manifests.yaml, kustomize/base/*, kustomize/components/.../otel-collector.yaml
Retagged images v0.10.2→v0.10.4 across bases; release bundle port/probe/selector edits; OTEL init/gateway image digest updates; loadgenerator RATE env added.
Cloud Build & Skaffold
cloudbuild.yaml, skaffold.yaml
Skaffold image bumped in Cloud Build; skaffold.yaml adds multi‑platform targets and enables local Docker CLI + BuildKit.
Documentation & minor text fixes
README.md, docs/*, kustomize/components/*/README.md, docs/releasing/license_header.txt, release/istio-manifests.yaml
Added demo links and microservice guide; clarified GKE/dev guide and kubectl usage; small README/typo and copyright year updates.
Java service (adservice)
src/adservice/Dockerfile, src/adservice/build.gradle, src/adservice/gradle/wrapper/*, src/adservice/gradlew*
Multi‑arch Dockerfile and runtime base updates; protobuf/grpc/Jackson/protoc/tooling version bumps; Gradle wrapper invocation changed to run the wrapper jar.
.NET cartservice
src/cartservice/src/*, src/cartservice/tests/*
Multi‑arch Dockerfiles; project target net10.0 and dependency upgrades; AlloyDBCartStore: added logging, read‑before‑write and single upsert (INSERT ... ON CONFLICT DO UPDATE); standardized error messages.
Go services & build tooling
src/checkoutservice/*, src/frontend/*, src/productcatalogservice/*, src/shippingservice/*, src/*/go.mod, `src/*/(main
server).go, src/frontend/validator/validator.go`
Node services (currency, payment)
src/currencyservice/*, src/paymentservice/*
Image and dependency upgrades; tracing initialization migrated to OpenTelemetry NodeSDK with OTLP exporter and sdk.start() when ENABLE_TRACING == "1".
Python services
src/emailservice/*, src/recommendationservice/*, src/shoppingassistantservice/*
Dockerfiles platform‑aware and alpine/apk conversions; PYTHON_* env flags added; broad requirements/lock upgrades; Stackdriver profiler startup commented/disabled in some services.
Loadgenerator
src/loadgenerator/*, kubernetes-manifests/loadgenerator.yaml, helm-chart/templates/loadgenerator.yaml, kustomize/base/loadgenerator.yaml
Platform‑aware image updates; added RATE env and updated Locust entrypoint to use -r "${RATE:-1}"; init image digest updates.
Misc multi‑arch Dockerfile updates
src/*/*Dockerfile (various)
Widespread migration to --platform builds, TARGETOS/TARGETARCH args, runtime base image upgrades, and build/publish adjustments across services.
Renovate config
​.github/renovate.json5
Replaced fileMatch with managerFilePatterns regex entries for specified managers.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor Client
  participant Service as Node service
  participant SDK as NodeSDK (otel/sdk-node)
  participant Exporter as OTLP gRPC Exporter
  participant Collector as OTLP Collector

  Note over Service: Startup when ENABLE_TRACING == "1"
  Service->>SDK: construct NodeSDK(resource, exporter)
  SDK->>Exporter: configure OTLP exporter
  Service->>SDK: sdk.start()
  SDK->>Exporter: initialize connection

  Note over Client,Service: Request handling
  Client->>Service: RPC/HTTP request
  Service->>SDK: create spans
  SDK->>Exporter: export spans
  Exporter->>Collector: send OTLP traces
Loading
sequenceDiagram
  autonumber
  participant Helm as Template engine
  participant Values as Chart values
  participant Output as Rendered YAML

  Helm->>Values: evaluate .Values.securityContext.enable
  alt securityContext enabled
    Helm->>Output: render securityContext {...}
    Helm->>Values: evaluate .Values.seccompProfile.enable
    alt seccompProfile enabled
      Helm->>Output: render seccompProfile inside securityContext
    else
      Helm-->>Output: skip seccompProfile
    end
  else
    Helm-->>Output: omit securityContext and seccompProfile
  end
Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

Poem

I nibble logs and chase new tags,
Seccomp snug where contexts flag.
Spans hop out on OTLP streams,
Multi‑arch burrows power dreams.
Hop, build, deploy — a rabbit's delight! 🐇✨

🚥 Pre-merge checks | ❌ 3
❌ Failed checks (2 warnings, 1 inconclusive)
Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description contains only an empty template with no substantive content in any section, providing no context about the changes, rationale, testing, or dependencies. Complete all template sections with detailed information: explain the dependency upgrades, describe testing performed, and clarify any deployment impacts or breaking changes.
Docstring Coverage ⚠️ Warning Docstring coverage is 9.09% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check ❓ Inconclusive Title is vague and generic, using 'merge latest upstream main' without clearly describing the primary changes or scope of the work. Provide a more specific title that summarizes the main changes, such as 'Upgrade dependencies and modernize container images' or 'Update Docker images, Java, Node.js, and related tooling'.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

renovate-bot and others added 30 commits August 24, 2026 16:44
Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.83.1 to 1.83.2.
- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.83.1...v1.83.2)

---
updated-dependencies:
- dependency-name: google.golang.org/grpc
  dependency-version: 1.83.2
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants